Skip to content

Conversation

@vtnerd
Copy link
Contributor

@vtnerd vtnerd commented Jun 29, 2025

There is one small issue with #647 - it's impossible to do a proper "lookahead" when doing an "import" or "recovery" scan. Basically, the client is forced to do constant /import_wallet_request calls every time it observes a new subaddress in one of the /get... endpoints. This isn't really practical.

Instead, a lookahead and height field is added to the /import_wallet_request endpoint, and the server asynchronously returns failure if the subaddress max is reached in one of the /get... endpoints.

#647 should still be merged as those endpoints are useful for new wallets where no re-scanning is required.

This requires #647 as the address_meta object is defined in that PR but not this one.

I'm also going to update the get_subaddrs endpoint with a new field to return max subaddresses permitted, but lets get #647 merged first.

@vtnerd vtnerd force-pushed the update/lws_subddr_lookahead branch 2 times, most recently from 90a320c to ecf2bc3 Compare July 3, 2025 17:27
@vtnerd vtnerd force-pushed the update/lws_subddr_lookahead branch from ecf2bc3 to aaee3a1 Compare December 4, 2025 19:12
@vtnerd
Copy link
Contributor Author

vtnerd commented Dec 4, 2025

Force pushed the latest recommended spec for lookahead. This is the one in use my monero-lws.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant